Add intrinsic-test alias and set sample rate - #159898
Conversation
|
|
This comment has been minimized.
This comment has been minimized.
|
I just realized that we didn't update the In the meantime, let's check how this affects the duration on aarch64: @bors try jobs=aarch64-gnu |
This comment has been minimized.
This comment has been minimized.
Add intrinsic-test alias and set sample rate try-job: aarch64-gnu
|
Seems like on |
Yeah we forgot about |
|
@Kobzol I think it's ready to get merged now :). Thank you! |
|
Thanks! You can r=me once PR CI is green :) @bors delegate |
|
@bors r=Kobzol |
… r=Kobzol Add intrinsic-test alias and set sample rate This PR adds intrinsic-test alias and set sample rate. It sets sample percentage to 100% . r? @Kobzol
Rollup of 17 pull requests Successful merges: - #159014 ([rustdoc] Do not take `doc(cfg())` into account when filtering doctests) - #159130 (a bit optimize four-digit chunks in integer formatting) - #159592 (core: implement bounded random sampling) - #159898 (Add intrinsic-test alias and set sample rate) - #158247 (hermit/fs: Return `unsupported()` instead of `from_raw_os_error(22)`) - #158649 (Hermit: fix `readdir()` ) - #159049 (Avoid ICE in From/TryFrom cast suggestion when encountering HRTBs) - #160053 (test: add test suite for the 85681 issue) - #160087 (Add regression test for nested associated-type projection ICE) - #160090 (rustc_resolve: Further reduce mutability in resolver) - #160099 (Resolver: split module resolutions into local and external resolutions) - #160106 (Add suggestions for `must_implement_one_of`) - #160117 (Remove unnecessary format usage) - #160134 (Work around Wine bug 60084 by calling WSAStartup at most once) - #160142 (bootstrap: remove use-lld config alias) - #160148 (Rename `errors.rs` file to `diagnostics.rs` (15/N)) - #160151 (Mark a doctest as requiring unwinding)
… r=Kobzol Add intrinsic-test alias and set sample rate This PR adds intrinsic-test alias and set sample rate. It sets sample percentage to 100% . r? @Kobzol
|
@bors p=6 |
This comment has been minimized.
This comment has been minimized.
Add intrinsic-test alias and set sample rate This PR adds intrinsic-test alias and set sample rate. It sets sample percentage to 100% . r? @Kobzol
|
Auto build was cancelled. Cancelled workflows: The next pull request likely to be tested is #160238. |
This comment has been minimized.
This comment has been minimized.
Add intrinsic-test alias and set sample rate This PR adds intrinsic-test alias and set sample rate. It sets sample percentage to 100% . r? @Kobzol
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test for ed50094 failed: CI. Failed job:
|
|
@bors retry |
This comment has been minimized.
This comment has been minimized.
Add intrinsic-test alias and set sample rate This PR adds intrinsic-test alias and set sample rate. It sets sample percentage to 100% . r? @Kobzol
|
@bors yield To give priority to the LLVM update PR. |
|
Auto build was cancelled. Cancelled workflows: The next pull request likely to be tested is #158734. |
|
@bors p=6 Scheduling |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 922325b (parent) -> fc0f51f (this PR) Test differencesShow 2 test diffs2 doctest diffs were found. These are ignored, as they are noisy. Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard fc0f51f5ca44d4586b0e309181382608cd6a8441 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (fc0f51f): comparison URL. Overall result: ❌✅ regressions and improvements - no action needed@rustbot label: -perf-regression Instruction countOur most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.
Max RSS (memory usage)Results (secondary 1.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary 0.3%, secondary -2.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeThis perf run didn't have relevant results for this metric. Bootstrap: 490.333s -> 491.246s (0.19%) |
View all comments
This PR adds intrinsic-test alias and set sample rate.
It sets sample percentage to 100% .
r? @Kobzol